Skip to content

Conversation

@denniskigen
Copy link
Member

Update UUIDs for duplicate concepts in drugs_concepts-core_demo.csv to match CIEL external IDs, preventing DuplicateConceptNameException errors during initialization.

Updated concepts:

  • Amikacin: 77413 (CIEL:77413)
  • Benzyl Benzoate: 72075 (CIEL:72075)
  • Docusate Sodium: 75121 (CIEL:75121)
  • Heparin: 77413 (CIEL:77

Pastebin log indicating DuplicateConceptNameException errors during initialization https://pastebin.com/raw/s7UgB9Xk

@dkayiwa
Copy link
Member

dkayiwa commented Nov 27, 2025

Update UUIDs for duplicate concepts in drugs_concepts-core_demo.csv to
match CIEL external IDs, preventing DuplicateConceptNameException errors
during initialization.

Updated concepts:
- Amikacin: 77413 (CIEL:77413)
- Benzyl Benzoate: 72075 (CIEL:72075)
- Docusate Sodium: 75121 (CIEL:75121)
- Heparin: 77413 (CIEL:77

Pastebin log indicating DuplicateConceptNameException errors during initialization https://pastebin.com/raw/s7UgB9Xk
@denniskigen denniskigen force-pushed the fix/duplicate-drug-concepts branch from 1103a33 to eadf9a5 Compare November 27, 2025 13:46
@dkayiwa
Copy link
Member

dkayiwa commented Nov 27, 2025

@dkayiwa
Copy link
Member

dkayiwa commented Nov 27, 2025

@denniskigen @VeronicaMuthee i am merging this because it fixes the duplicate errors.
We can use a different pull request to fix the remaining errors like this: java.lang.IllegalArgumentException: No drug concept found for '062fc7f6-3d11-409a-96f7-a60cd8a71a36'

@dkayiwa dkayiwa merged commit d479720 into main Nov 27, 2025
1 check passed
@denniskigen
Copy link
Member Author

Thanks, @dkayiwa. Are you able to test this one too?

@denniskigen
Copy link
Member Author

denniskigen commented Nov 27, 2025

Now this is what we have: gist.github.com/dkayiwa/ed7167a5670e1abbfa5e0490e4634ee1

Also: Dear reader, the remaining issue was fixed in a subsequent force push.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants